scalebutton: Don't derive from GtkButton

Make GtkScaleButton a widget that has a toggle button
as a child, just like all the other button widgets now.
The immediate benefit of this arrangement is to avoid
the "double focus" problem when we pop up the popup.

Update accessible, demos and tests to match.

Merge branch 'wip/matthiasc/can-focus' into 'master'

Implement new focus behavior

See merge request GNOME/gtk!1652

Mention focus changes in the migration guide

Small documentation tweaks

Update the docs for GtkWidget:can-focus and :focus-on-click.

Move focus to the parent if a popover is dismissed

This is better than just dropping focus altogether,
and is expected behavior in most cases.

Update a11y test results

a11y: Fix focusable state

The :can-focus property is no longer very useful to
give an indication of what is focusable, since it is
TRUE for almost all widgets now. Patch things up
to by looking at known widget types.

Fix the tools tests to not set can-focus

Disable focus-chain test that depends on the environment

The color editor shows a color picker button only if it
finds a suitable implementation, which it does not in ci.
So disable the focus-chain test for page 3.

color editor: Make the sample non-focusable again

This is one of the situations, where can-focus can still
be used to tweak focus behavior of leaf widgets. Color
swatches are focusable by default to allow selecting colors
with the keyboard. But when used as color samples, they
should not take focus.

Remove focusable-container tests

Like the previous commit, these tests were relying
on setting :can-focus to make an unsuspecting container
focusable, which does not work anymore.

Remove the focus test

This test was expecting to make existing widgets like
GtkBox focusable by setting :can-focus. That just doesn't
work anymore.

The focus chain testing that is done here is already
better covered by test-focus-chain, so lets just remove this.

Fix up focus chain result

The notebook grab_focus change in the previous commit made
backwards tabbing work as expected, and thereby changed the
output of one of the focus-chain tests.

Reinstate expected focus behavior

After the :can-focus change in the previous commit, widgets
need to set suitable focus and grab_focus implementations
to implement the desired focus behavior.

This commit does that for all widgets.

Change the default value of GtkWidget:can-focus

Make widgets can-focus by default, and change the semantics
of can-focus to be recursive . If it is set to FALSE, focus
can not enter the widget or its descendents at all anymore.

This commit temporarily breaks focus behavior of widgets
that did not expect to receive focus.

widget: Add common focus vfunc implementations

Privately export a number of implementations for the focus
and grab_focus vfuncs that cover many common cases.

gizmo: Allow passing changing focus behavior

We need this in popovers. Maybe it could be done better
by defining one-off custom widgets.

widget: Make the :has-focus property readonly

The only place where this should be set is when making
a widget the focus-widget of a window. We still keep
the property around in readonly form, since there are
a few places where we rely on property notification
for it.

testsuite: Stop setting :has-focus

This is not necessary, whenever we want to set the initial
focus in a ui file, we can set GtkWindow:focus-widget.

widget: Drop the :is-focus property

This property doesn't carry any new information compared
to GtkWindow:focus-widget. We still keep the gtk_widget_is_focus
getter, as a convenient shortcut.

reftests: Stop using can-focus

a11y tests: Stop using can-focus

testsuite: Stop using can-focus

inspector: Drop more uses of can-focus

inspector: Remove can-focus from ui files

Stop setting can-focus in ui files

None of these settings are necessary.

window: Make set_focus equal to grab_focus

Make gtk_window_set_focus call gtk_widget_grab_focus internally.
This means that set_focus can now end up putting the focus on
a child of the passed-in widget, and makes the focus-widget
property work for setting initial focus to (the child of) an
entry in a ui file.

root: Reorganize focus handling

Make :focus-widget a GtkWindow property and add vfuncs
to the GtkRoot interface instead of the property.

Add more focus chain tests

Test page 2 and 3 of widget-factory as well, and try other directions.

Fix running focus-chain tests in ci

The widget-factory ui files require geettings
(for the color chooser), so set GSETTINGS_SCHEMA_DIR.

notbook: Fix a bug in focus handling

After the header widget was introduced, focus would get
stuck in a loop between actions and tabs.

This could be seen in the notebook on page 3 of
widget-factory.

coloreditor: Draw a focus around the color plane

Not doing it was just an oversight.

colorplane: Set a css name

Every widget should have one.

Merge branch 'wip/xdg-popup-move' into 'master'

Wayland popup moving

See merge request GNOME/gtk!1017

wayland: Move transient-for field to GdkWaylandToplevel

A toplevel will only ever be transient-for to another toplevel, and only
a toplevel will ever be transient-for, so move the field into the
GdkWaylandToplevel, and make it a pointer to another GdkWaylandToplevel.

wayland: Add surface to toplevel list only if toplevel

It was using another check (has parent) to do this, but now we have a
much more obvious way of creating surfaces, so we can use the type
directly.

wayland: Remove own pointer to the popup parent

It's there already as the GdkSurface::parent, no need to duplicate.

wayland: Move toplevel/popup/drag surface definitions higher up

We them up there, so that code higher up compared to where they are
defined now can make use of them. Also add a few macros for type
checking and casting.

wayland: Move popups with xdg_popup.reposition

The third version of xdg-shell introduces support for explicit popup
repositioning. If available, make use of this to implement popup
repositioning.

Note that this does *NOT* include atomic parent-child state
synchronization. For that,
https://gitlab.freedesktop.org/wayland/wayland-protocols/issues/13 will
be needed.

This currently uses my own fork of wayland-protocols which adds meson
support, so that we can use it as a subproject. Eventually when
wayland-protocols' meson support lands upstream, we should change it to
point there.

Silence some meson warnings while at it to make CI happy.

This also bumps the glib requirement, since g_warning_once() is used.

build: Change wrap revisions to use master instead of origin/master

Using origin/master crashes meson.

Merge branch 'Gtk4_blue_focus_rings' into 'master'

Blue outlines / focus rings

See merge request GNOME/gtk!1629

Blue outlines / focus rings

- introduce two new colors: $focus_border_color for focused / outlined elements and $_coloured_focus_border_color for focused / outlined elements with a colored background color, like suggested/destructive buttons or selected elements
- set outline / focus color, offset and style accordingly for all widgets
- adapt entry focus color

Merge branch 'render-node-types' into 'master'

Turn GskRenderNode into a fundamental type

See merge request GNOME/gtk!1649

Properly annotate the render node constructors

Now that the GskRenderNode subclasses are recognised as proper
sub-types, we can annotate the constructors with their type. The C API
remains the same.

Add get_type() functions for GskRenderNode subclasses

The introspection scanner tries to match a type name with a get_type()
function, in order to correctly identify a type as a class.

If the function is not available, we have two choices:

 - add some special case inside the introspection scanner, similar to
   how GParamSpec subclasses are handled in GObject
 - add a simple get_type() function

The latter is the simplest approach, and we don't need to change that
much, since we still register all render nodes at once.

Do not install GSK headers for missing backends

Broadway and Vulkan renderers are optional; if GTK hasn't been built
with their GSK renderers, we should not install their headers.

Improve the gtktypefuncs.inc generator

Remove the plug/socket exception, and add exceptions for non-X11
windowing systems.

Additionally, speed up the file generation by avoiding string
concatenation in Python.

Add missing transfer annotation

Turn GskRenderNode into a derivable type

Language bindings—especially ones based on introspection—cannot deal
with custom type hiearchies. Luckily for us, GType has a derivable type
with low overhead: GTypeInstance.

By turning GskRenderNode into a GTypeInstance, and creating derived
types for each class of node, we can provide an introspectable API to
our non-C API consumers, with no functional change to the C API itself.

Add macro for exported variables

The logic is based on the similar macro in GLib, but with the
appropriate GDK symbols.

Merge branch 'ci-meson-bump' into 'master'

ci: Update the version of Meson in our Docker image

See merge request GNOME/gtk!1651

ci: Update the version of Meson in our Docker image

Needed for !1017.

Merge branch 'matthiasc/for-master' into 'master'

Remove an unused backend includes

See merge request GNOME/gtk!1648

Merge branch 'imcontextsimple-drop-ifdefs' into 'master'

impcontextsimple: Drop ifdef'ed platform dependencies

See merge request GNOME/gtk!1647

Remove an unused backend includes

These includes are just leftovers.

Every time where we do an #ifdef GDK_WINDOWING... in gtk/,
something went wrong. Don't do it needlessly.

impcontextsimple: Drop ifdef'ed platform dependencies

The proper way to do this would be to adapt the tables
to have the right data for the platform. Since 4.0 is
a new start in many ways, lets clean this up.

Merge branch 'private-x11-types' into 'master'

Do not parse all GDK-X11 source files

See merge request GNOME/gtk!1646

Rename gdkx11keys.h

The header is now private, so it should follow the same naming scheme
for private GDK-X11 headers.

Hide GdkX11Keymap's GType function

It's not a public object.

Do not parse all GDK-X11 source files

We don't need all of them, only the ones that contain public API. This
allows us to reduce the chance of a stray symbol getting incorrectly
added to the introspection data.

Merge branch 'attribute-docs' into 'master'

entry: Document buildable support

See merge request GNOME/gtk!1643

entry: Document buildable support

entry: Add <attributes> support

Make entries support the same <attributes> syntax
as labels.

Closes: #1335

Move label attribute parser code

We want to reuse the parser for <attributes> in several
widgets, so move it to a shared place.

Merge branch 'cross-postinstall' into 'master'

meson: Don't execute post-install.py if cross compiling

See merge request GNOME/gtk!1636

Merge branch 'fix-event-match' into 'master'

shortcuttrigger: Don't trigger on key releases

See merge request GNOME/gtk!1642

Merge branch 'wip/chergert/fix-shortcut-param-order' into 'master'

widget: fix parameter ordering to match implementation

Closes #2586

See merge request GNOME/gtk!1641

shortcuttrigger: Don't trigger on key releases

This broke when the event type check in gdk_key_event_matches
was removed and replaced by a precondition that accepts both
key press and release events.

Add the check in gtk_keyval_trigger_trigger instead.

widget: fix parameter ordering to match implementation

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/2586

Replace most uses of <Primary> with <Control>

The only place where we are still using <Primary> is
in tests, to ensure we keep parsing it. Otherwise,
<Control> is now the preferred syntax.

Merge branch 'keymap-rework-2' into 'master'

Keymap rework 2

See merge request GNOME/gtk!1635

reftests: Comment out a test that fails only in ci

Fixing such only-in-ci failures is just too hard,
unfortunately.

Update migration guide

Rewrite the sections about keymaps and menus, and add
hints about modifiers.

accelgroup: Small documentation updates

Mention GtkShortcutTrigger, and remove references
to <Release>.

Reshuffle keymap docs

Since GdkKeymap api is on longer public, move relevant
documentation to the long description.

Drop gtk_accelerator_set_default_mod_mask

Our new approach to modifiers works with a fixed set,
there is really no need to customize the modifier
masks if the backends are all supposed to deliver
the same modifiers.

gdk: Remove gdk_keymap_get_modifier_mask

This function is no longer public and no longer used.

Remove uses of modifier intents in gdkevents.c

This removes the use of the context menu and shift group
intents in gdkevents.c. If it turns out to be important,
we need to introduce vfuncs for gdk_event_triggers_context_menu
and gdk_event_matches.

display: Remove gdk_display_get_modifier_mask

This is not used anymore.

gtk: Drop gtk_widget_get_modifier_mask

This is not used anymore.

gtk: Stop using modifier intents

Reviewing the existing settings, the only backend with
some differences in the modifier intent settings is OS X,
and we would rather have that implemented by interpreting
the existing modifiers in the appropriate way.

                X11      Wayland  Win32    OS X

primary         ctrl     ctrl     ctrl     mod2
mnemonic        alt      alt      alt      alt
context menu    -        -        -        ctrl
extend sel      shift    shift    shift    shift
modify sel      ctrl     ctrl     ctrl     mod2
no text         alt|ctrl alt|ctrl alt|ctrl mod2|ctrl
shift group     varies   -        -        alt

GTK now uses the following modifiers:

primary         ctrl
mnemonic        alt
extend sel      shift
modify sel      ctrl
no text         alt|ctrl

The context menu and shift group intents were not used
in GTK at all.

Update tests to no longer expect <Primary> to roundtrip
through the accelerator parsing and formatting code.

keymap: Remove virtual modifier mapping functions

These are not used anymore.

Drop the vfuncs and their implementations in the
broadway, x11, wayland and win32 backends as well.

gdk: Make GdkKeymap a private api

We have replacement apis in GdkDevice and GdkDisplay.

Merge branch 'misc-meson-fixes' into 'master'

Reduce useless relinking on configure and fix check for buildtype arguments

See merge request GNOME/gtk!1614

Update Dutch translation

imcontextsimple: Disable some win32-only code

This code needs to be redone differently, since keymaps are no
longer going to be exposed. There should really not be this much
ifdef-ed backend-specific code here anyway. Or any, really.

gdk: Clean up GdkModifierType

Remove MOD2..MOD5. Backends are expected to just set
the named modifiers.

gdk: Redo key events

Add all of the keyboard translation results in the key event,
so we can translate the keyboard state at the time the event
is created, and avoid doing state translation at match time.

We actually need to carry two sets of translation results,
since we ignore CapsLock when matching accelerators, in
gdk_event_matches().

At the same time, drop the scancode field - it is only ever
set on win32, and is basically unused in GTK.

Update all callers.

gtk: Remove handling of virtual modifiers

These are going away. We expect to just have
named modifiers in modifier masks now, so we
longer juggle 'real' and 'virtual' modifiers.

events: Stop doing elaborate virtual modifier handling

These are going away in the GdkModifierType cleanup.
Just compare the modifiers we got.

broadway: Stop using MOD2..MOD5

These are going away in the GdkModifierType cleanup.

win32: Stop using MODx modifiers

The win32 backend is using GDK_MOD2_MASK for AltGr,
so define GDK_MOD2_MASK locally to keep this working,
but remove any mention of GDK_MOD3_MASK,...,GDK_MOD5_MASK.

Update Ukrainian translation

Update Ukrainian translation

Merge branch 'fno-common' into 'master'

Add a missing extern, to fix building with -fno-common

See merge request GNOME/gtk!1631

wayland: Stop setting MOD2..MOD5

These are going away in the GdkModifierType cleanup.

gdk: Remove reserved bit from GdkModifierType

This really has no purpose and obscures the header.

gtk: Stop using GDK_MODx_MASK

These are never used in practice, and we never want to
see them in the UI, so stop supporting them. This is
in preparation for cleaning up GdkModifierType.

Rename GDK_MOD1_MASK to GDK_ALT_MASK

We've hardcoded Mod1 = Alt for a long time, there is
no need to keep the confusing naming around anymore.

tests: Stop using gtk_accelerator_set_default_mod_mask

This is not doing anything useful here.